theme > search results: revisions to padding, drop bg from app results

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/904

diff --git a/data/theme/gnome-shell-sass/widgets/_message-list.scss b/data/theme/gnome-shell-sass/widgets/_message-list.scss
index a51b07d65c54e7616438030844d3f8052dfe05a3..56541ba345b3f5bb45f406d7adc2e66a401075d1 100644 (file)
--- a/data/theme/gnome-shell-sass/widgets/_message-list.scss
+++ b/data/theme/gnome-shell-sass/widgets/_message-list.scss
@@ -59,9 +59,11 @@
   // close button
   .message-close-button {
     color: lighten($fg_color, 15%);
-    &:hover { color: darken($fg_color, 10%);}
+    &:hover { color: if($variant=='light', lighten($fg_color, 30%), darken($fg_color, 10%)); }
+    &:active { color: if($variant=='light', lighten($fg_color, 40%), darken($fg_color, 20%)); }
   }
 
+
   // icon container
   .message-icon-bin {
     padding: $base_padding;

diff --git a/data/theme/gnome-shell-sass/widgets/_search-results.scss b/data/theme/gnome-shell-sass/widgets/_search-results.scss
index 7b1ae0a152e5e59f849cdac24775369a0ae694bc..15a9ee0a09fe7cbc3f442bcfaf467babc83b4c0c 100644 (file)
--- a/data/theme/gnome-shell-sass/widgets/_search-results.scss
+++ b/data/theme/gnome-shell-sass/widgets/_search-results.scss
@@ -7,35 +7,33 @@
 
 // search results sections "the boxes"
 .search-section {
-  background-color: transparentize(lighten($osd_bg_color, 5%), 0.2);
-  border-radius: $modal_radius+3;
-  border: 1px solid $osd_outer_borders_color;
-  box-shadow: 0 2px 4px 0 $shadow_color;
-  text-shadow: 0 1px if($variant == 'light', rgba(255,255,255,0.2), rgba(0,0,0,0.2));
-  color: $osd_fg_color;
-  padding: $base_padding * 3;
-  margin: $base_margin;
-  spacing:$base_spacing * 2;
-
-  // hide separator
-  &:first-child .search-section-separator,
-  &:last-child .search-section-separator {
-    background-color: transparent;
-  }
-
+  spacing: $base_margin * 2;
+  padding:0 !important;
+  margin:0 !important;
+  background-color:transparent;
+  box-shadow:none;
+  border:none;
   // separator
   .search-section-separator { 
     // margin-top: $base_padding * 2;
     // height: 1px;
     // background-color: $osd_outer_borders_color;
-    height: 0px;
+    height: 0;
     background-color: transparent;
   }
+}
 
-  // content
-  .search-section-content {
-    spacing: 0;
-  }
+// content
+.search-section-content {
+  background-color: transparentize(lighten($osd_bg_color, 5%), 0.2);
+  border-radius: $modal_radius+3;
+  border: 1px solid $osd_outer_borders_color;
+  box-shadow: 0 2px 4px 0 $shadow_color;
+  text-shadow: 0 1px if($variant == 'light', rgba(255,255,255,0.2), rgba(0,0,0,0.2));
+  color: $osd_fg_color;
+  padding: $base_padding * 3;
+  margin: $base_margin 0;
+  spacing: 0;
 }
 
 // "no results" text
@@ -78,7 +76,7 @@
 .search-provider-icon {
   @extend %icon_tile;
 
-  padding: 0;
+  padding: $base_padding;
   spacing: 0;
   margin-right: $base_margin * 2;
 
@@ -120,6 +118,8 @@
   padding: $base_padding;
   color: $osd_fg_color;
 
+  border-radius: $base_border_radius + 2px !important;
+
   &:focus,
   &:selected,
   &:hover {